Skip to content

[3.9] bpo-45624: make test_graphlib not depend on the iteration order of sets (GH-29233) #29292

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 28, 2021

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Oct 28, 2021

the current test depended on integer sets being iterated on in a certain
fixed order. That order is different on PyPy (insertion based) and could
change in CPython in the future in theory. Make the test robust against
a different iteration order by sorting.
(cherry picked from commit 7401694)

Co-authored-by: Carl Friedrich Bolz-Tereick [email protected]

https://bugs.python.org/issue45624

…ts (pythonGH-29233)

the current test depended on integer sets being iterated on in a certain
fixed order. That order is different on PyPy (insertion based) and could
change in CPython in the future in theory. Make the test robust against
a different iteration order by sorting.
(cherry picked from commit 7401694)

Co-authored-by: Carl Friedrich Bolz-Tereick <[email protected]>
@bedevere-bot bedevere-bot added the tests Tests in the Lib/test dir label Oct 28, 2021
@miss-islington
Copy link
Contributor Author

@cfbolz and @ambv: Status check is done, and it's a failure ❌ .

@miss-islington
Copy link
Contributor Author

@cfbolz and @ambv: Status check is done, and it's a success ✅ .

@ambv ambv merged commit 67a1abb into python:3.9 Oct 28, 2021
@miss-islington miss-islington deleted the backport-7401694-3.9 branch October 28, 2021 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants